-
Notifications
You must be signed in to change notification settings - Fork 94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for multiple Example Tables with tags #161
Conversation
@phil-davis LGTM |
This looks like the place that shows this is supported in Cucumber-Gherkin
and selects various combinations of the tagged examples tables. So this |
@ciaranmcnulty I don't know who to ping for review (and hopefully approval) of this. |
Feature sounds correct to me; will review the code when I get a chance |
@ciaranmcnulty any chance to progress this? |
@ciaranmcnulty @stof @everzet any chance for someone to review this? |
@ciaranmcnulty @stof @everzet any chance for someone to review this? |
@ciaranmcnulty @stof @everzet any chance for someone to review this? |
I see some activity in @ciaranmcnulty @stof @everzet can anyone progress this? Or tell us that it is not going to be accepted? Or give any feedback please. |
@ciaranmcnulty @stof @everzet pinging again. We had a situation today where this would be useful - putting different tags on each example. |
2ed729d
to
5fdf330
Compare
I rebased just now to get fresh CI against current master. @ciaranmcnulty @stof I see that a PR was merged 2 days ago. Any chance to review this one? |
@ciaranmcnulty @stof is there any chance that this will be accepted? We would like to be able to use this functionality, and it seems to be part of "Gherkin standard" (if that is a thing!) |
@phil-davis Afaik @pamil took over maintenance of Behat recently. I have already pointed out this PR to him, but I guess he's still in the middle of getting an overview of all the open tasks. |
5fdf330
to
d2e0bf9
Compare
d2e0bf9
to
476cc51
Compare
I rebased this just now to get fresh CI. It would be really nice if we could get a release of this, and whatever else is waiting in |
@ciaranmcnulty and whoever else is managing this repo. Is there any chance that this will ever be merged and released? It would be nice to have. We want to be able to tag Scenario Examples - e.g. see my comment owncloud/ocis-reva#282 (comment) Please be honest and say if it will never be merged. Then we can close it and stop hoping. |
476cc51
to
55ee35c
Compare
@stof I see that you have merged some things last month. Any chance for this to be looked at? |
/cc @pamil as you announced you are taking over the maintenance of Behat. |
Thanks @dpakach, sorry it took >1yr! |
Thank you very much @ciaranmcnulty |
Description
Add support for multiple example table with tags on scenario outlines
This is a PR is addition to #119
Related Issue #117
Example:
Context
Having this allows us to use tags to run only small set of examples from the table on a big test suite.
for eg. in above example we can use
@smokeTest
tag filter to run only a basic example of the scenario. but If we want to run all the examples we can basically run it without any tags.